Skip to content

DON'T MERGE - Runner test#5693

Closed
europaul wants to merge 2 commits intolf-edge:masterfrom
europaul:runner-test
Closed

DON'T MERGE - Runner test#5693
europaul wants to merge 2 commits intolf-edge:masterfrom
europaul:runner-test

Conversation

@europaul
Copy link
Copy Markdown
Contributor

this PR is only here to test the new VPS runners

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 29.45%. Comparing base (2281599) to head (f3c0cb3).
⚠️ Report is 346 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5693      +/-   ##
==========================================
+ Coverage   19.52%   29.45%   +9.92%     
==========================================
  Files          19       18       -1     
  Lines        3021     2417     -604     
==========================================
+ Hits          590      712     +122     
+ Misses       2310     1554     -756     
- Partials      121      151      +30     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@europaul
Copy link
Copy Markdown
Contributor Author

/rerun red

@europaul europaul force-pushed the runner-test branch 2 times, most recently from 4e14708 to 83c6a71 Compare March 20, 2026 16:14
zedagentCtx.fatalFlag = *zedagentCtx.fatalPtr
zedagentCtx.startTime = time.Now()

// nothing ever changes
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this change have to do with the runners?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's just some change in the source code to trigger the build workflow
this whole PR is for test purposes only and should not be merged

Copy link
Copy Markdown
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Run tests

@europaul europaul changed the title Runner test DON; T MERGE - Runner test Mar 20, 2026
@europaul europaul changed the title DON; T MERGE - Runner test DON'T MERGE - Runner test Mar 20, 2026
@github-actions github-actions bot requested a review from eriknordmark March 20, 2026 18:28
europaul and others added 2 commits March 25, 2026 19:30
to trigger the right workflows

Signed-off-by: Paul Gaiduk <paulg@zededa.com>
When building EVE with HV=k, the Makefile includes additional packages
(kube, external-boot-image) and builds pillar with build-k.yml instead
of build.yml. Without a separate package build step for HV=k, the eve
job fails because these packages are missing from the linuxkit cache.

Add an amd64/generic/k entry to the packages matrix so that k-specific
packages are built and cached separately. Update all cache keys to
include the hv dimension (using 'default' for non-k builds and 'k' for
the kubernetes variant).

Signed-off-by: Paul Hendry <phendry@zededa.com>

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Paul Gaiduk <paulg@zededa.com>
@rene
Copy link
Copy Markdown
Contributor

rene commented Mar 27, 2026

Closing since #5709 is merged. The new runners are in place.

@rene rene closed this Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants